-
-
Notifications
You must be signed in to change notification settings - Fork 23
Dockerfile for PET2BIDS #359
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
|
Love it |
bendhouseart
approved these changes
Jul 14, 2025
Contributor
bendhouseart
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works and approved, but please update the Readme with something like the following as a last step.
Docker Usage
To build the docker file run:
docker build . -t openneuropet/pet2bids
Then mount and run your conversions on your input folders like so:
docker run --rm -it \ -v /path/to/dicoms:/dicoms:ro \ -v /path/to/outputdir:/out \ openneuropet/pet2bids dcm2niix4pet /dicoms --destination-path /out
Contributor
Author
|
Great, thank you. All done! |
bendhouseart
added a commit
that referenced
this pull request
Sep 9, 2025
* Bug Fix for sidecar flag on ecat2bids (#358) Fixes template json and sidecar issues posted by users Specifically fixes issues #352 and #354 posted on the github Rolling into version 1.4.4 * Dockerfile for PET2BIDS (#359) * Dockerfile for PET2BIDS * Updated ReadMe with docker usage * accept changes to avoid double [[ ]] solves #361 * Adding header information for ecat71 (#362) Courtesy of Mark Lubberink * Enhance README with spreadsheet usage instructions Added sections for single and multiple subject spreadsheets with links to templates and examples. * add dcm2niix binaries --------- Co-authored-by: Rami Hamati <[email protected]> Co-authored-by: Cyril Pernet <[email protected]> Co-authored-by: Granville Matheson <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I've made a Docker image for PET2BIDS.
Here is an example usage:
docker run --rm -it \ -v /path/to/dicoms:/dicoms:ro \ -v /path/to/outputdir:/out \ [container_name] dcm2niix4pet /dicoms --destination-path /out📚 Documentation preview 📚: https://pet2bids--359.org.readthedocs.build/en/359/